Skip to content

Tighten repository licensing#4

Open
zachyzissou wants to merge 2 commits intomainfrom
license-tighten-20260415
Open

Tighten repository licensing#4
zachyzissou wants to merge 2 commits intomainfrom
license-tighten-20260415

Conversation

@zachyzissou
Copy link
Copy Markdown
Owner

Adds the new restrictive source-visible LICENSE and README notice.

Copilot AI review requested due to automatic review settings April 15, 2026 21:38
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates repository-facing documentation to reflect a new restrictive, source-visible licensing stance.

Changes:

  • Adds a prominent license notice at the top of the README.
  • Introduces a new custom “source-visible” LICENSE with strong restrictions on use, copying, modification, and deployment.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
README.md Adds a top-of-file notice summarizing the new restrictive licensing terms and pointing readers to LICENSE.
LICENSE Adds a custom “source-visible” license text defining permitted viewing and prohibiting other uses without written permission.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread LICENSE Outdated
@@ -0,0 +1,32 @@
# SlurpNet Source-Visible License v1.0
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The license header names "SlurpNet" while the repository/project is "OmniFeed" (README title). If this license is intended for OmniFeed, rename the license title to match the project to avoid ambiguity about what work the terms apply to.

Suggested change
# SlurpNet Source-Visible License v1.0
# OmniFeed Source-Visible License v1.0

Copilot uses AI. Check for mistakes.
Comment thread LICENSE
Comment on lines +9 to +17
## No permission granted
Except for the limited right to view the source code, no other rights are granted.
Without prior written permission from Zach Gonser, you may not:
- use this code or any substantial portion of it in production or non-production systems,
- copy, reproduce, modify, adapt, translate, or create derivative works,
- distribute, sublicense, sell, lease, lend, publish, or transfer the code,
- deploy the code, or any modified version of it, for personal, internal, commercial, or public use,
- use the code to provide hosted or managed services,
- remove or alter copyright, attribution, or license notices.
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This repository includes third-party dependencies (e.g., vendored backend/node_modules/... with MIT/BSD licenses). The top-level LICENSE should explicitly clarify that it applies only to code you own, and that bundled third-party components remain under their original licenses (and that those notices are preserved), otherwise the current "No permission granted" section can be read as attempting to restrict rights that you can’t relicense.

Copilot uses AI. Check for mistakes.
Comment thread LICENSE
- remove or alter copyright, attribution, or license notices.

## Contributions
Unless explicitly agreed otherwise in writing, any contribution, patch, pull request, or suggestion submitted for this repository may be rejected and does not grant you any rights to this codebase.
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Contributions section says submissions "do not grant you any rights" but doesn’t state what license (if any) a contributor grants to the repository owner for accepted contributions. This can create ambiguity about whether maintainers may legally use/modify/redistribute contributed patches; consider adding an explicit inbound license/assignment statement (or referencing a CLA) for accepted contributions.

Suggested change
Unless explicitly agreed otherwise in writing, any contribution, patch, pull request, or suggestion submitted for this repository may be rejected and does not grant you any rights to this codebase.
Unless explicitly agreed otherwise in writing, any contribution, patch, pull request, or suggestion submitted for this repository may be rejected and does not grant you any rights to this codebase.
By submitting any contribution, patch, pull request, or suggestion to this repository, you grant Zach Gonser and contributors authorized by Zach Gonser a perpetual, irrevocable, worldwide, non-exclusive, royalty-free license to use, reproduce, modify, adapt, publish, distribute, sublicense, and incorporate that submission, in whole or in part, into this codebase and any related works.
You represent that you have the legal right to grant this license for any submission you make.

Copilot uses AI. Check for mistakes.
Comment thread README.md
Comment on lines +1 to +2
> **License Notice**
> This repository is source-visible only. No copying, modification, redistribution, deployment, or commercial use is permitted without prior written permission from Zach Gonser. See `LICENSE` for details.
Copy link

Copilot AI Apr 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new License Notice states that no copying/modification is permitted, but the same README later invites PRs and instructs contributors to run lint/tests. This is internally inconsistent and may confuse contributors; consider updating the Contributing section to reflect the new source-visible terms (e.g., clarify whether contributions/running locally are allowed under an explicit exception).

Copilot uses AI. Check for mistakes.
@zachyzissou
Copy link
Copy Markdown
Owner Author

@copilot review this PR for licensing correctness, unintended file changes, and any repo-specific risks.

Copy link
Copy Markdown

Copilot AI commented Apr 15, 2026

@copilot review this PR for licensing correctness, unintended file changes, and any repo-specific risks.

Reviewed and addressed the licensing risks in commit 5aa4f32: corrected the project name in LICENSE, added an explicit third-party license carve-out, clarified inbound contribution licensing, and aligned README contribution guidance with the source-visible terms. I also verified there were no unintended file changes beyond LICENSE and README.md, and backend tests passed. Screenshot: N/A (no UI changes).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants